Skip to content

add request context #372

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

colanconnon
Copy link

No description provided.

@urbandove
Copy link
Contributor

Looks great

I just wanted to note that @pizzapanther has a much larger update to the DRF serializer framework in pull request #326 which this PR should probably be merged with

Also - being that the graphene project has 2.0.1 with new supported types (specifically the Date type) - I would say its premature to bump the graphene-django version without supporting those new types (shameless plug for my pull request #335 ;) )

@colanconnon
Copy link
Author

@urbandove I agree this should be added along with that pr. Also, the version bump was just some debugging from me, I will revert that.

@colanconnon
Copy link
Author

colanconnon commented Jan 15, 2018

@urbandove I made a pr into that branch. If we get that merged I can close this. https://github.com/pizzapanther/graphene-django/pull/2

@syrusakbary
Copy link
Member

Closing this PR as #326 had the same fix and is already merged into master

@syrusakbary syrusakbary closed this Jun 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants